Skip to content

Adding pass2 steering files #1090

Merged
mgignac merged 1 commit intomasterfrom
mgignac-steering-files-pass2
Apr 3, 2025
Merged

Adding pass2 steering files #1090
mgignac merged 1 commit intomasterfrom
mgignac-steering-files-pass2

Conversation

@mgignac
Copy link
Collaborator

@mgignac mgignac commented Apr 1, 2025

Clones of the pass1 files with a few notable changes:

  • Recon steering files are added for skimmed and non-skimmed versions.
  • Vertex locations have been updated to those of 14629 (Moller run). This should be a better estimate than the previous (0,0,0), but still won't be perfect. Getting this correct will will require run-by-run information, and will be extracted from pass2 data.

sarahgaiser
sarahgaiser approved these changes Apr 1, 2025
View reviewed changes
Copy link
Collaborator

@sarahgaiser sarahgaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to update the MC steering files as well?

@bloodyyugo
Copy link
Contributor

bloodyyugo commented Apr 3, 2025

Can this be merged?

@mgignac
Copy link
Collaborator Author

mgignac commented Apr 3, 2025

Yes, was waiting for you to confirm the skimming was implemented correctly. Sounds like it is. I'll merge now.

@mgignac mgignac merged commit 02712f3 into master Apr 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@baltzell baltzell baltzell approved these changes

@sarahgaiser sarahgaiser sarahgaiser approved these changes

@bloodyyugo bloodyyugo Awaiting requested review from bloodyyugo

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mgignac @bloodyyugo @baltzell @sarahgaiser